Skip to content

Manifest of MAST notebooks to include in Fornax#162

Open
sedonaprice wants to merge 9 commits into
spacetelescope:mainfrom
sedonaprice:fornax-manifest
Open

Manifest of MAST notebooks to include in Fornax#162
sedonaprice wants to merge 9 commits into
spacetelescope:mainfrom
sedonaprice:fornax-manifest

Conversation

@sedonaprice

@sedonaprice sedonaprice commented May 11, 2026

Copy link
Copy Markdown
Contributor

This is a draft manifest of MAST notebooks to include in Fornax.

Steps required before ready to mark PR as non-draft/ready for review:

  • @ttdu : Sanity check: Descriptions? Thoughts on how to manage environments/dependencies?
  • Iterate with Fornax archive partners on format/required input to work with to-be-designed scripts in the fornax-images repo

@ttdu

ttdu commented May 12, 2026

Copy link
Copy Markdown
Collaborator

We should definitely use the notebook wrangler for managing dependencies.

I see you commented out the requirements but also point to the requirements.txt files. Is this the fornax standard?

@sedonaprice

sedonaprice commented May 12, 2026

Copy link
Copy Markdown
Contributor Author

There is no standard for Fornax, it's ad hoc and will depend on the infrastructure needs after iterating with our Fornax partners to design the "MAST Fornax ingest" scripts. I copied them in to cover the case where we need them, though we may just end up deleting that in favor of using "extra dependencies" and "environments".

Would you be able to take point on running the notebook wrangler for this set?

@sedonaprice

Copy link
Copy Markdown
Contributor Author

Format looks good from the Fornax side. @ttdu marking this as ready for review. I'll note the reqs file points to the astroquery fix fork as that has not yet been pulled to main, but I'd like to get this rolling.

@sedonaprice sedonaprice marked this pull request as ready for review May 28, 2026 15:28
Update to point to astroquery main, not the MAST beta branch, as astropy/astroquery#3598 has been merged in.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants